Auto merge of #4374 - behnam:new-init, r=alexcrichton

[cargo_new] Hint to use `cargo init` on existing dir

The `new` command always expects a non-existing path. The `init` command
always expects an existing path. Therefore, it makes sense to hint to
use `init` if the pre-condition to `new` is not satisfied.

Fixes <#4366>

Auto merge of #4375 - behnam:root-vs-repo, r=alexcrichton

[docs] Update language regarding repository vs package root

In the current documentation, there are many places with language that
assumes a *repository* (as in VCS) is the same as *package root* (the
directory where `Cargo.toml` sits).

With the new workspace features, this is far from true now, and the
inaccurate language makes it difficault for newbies or developers
without much familiarity with the cargo internals with authoring their
manifest files.

This diff tries to use the right terms for places any of these concepts
is referred to:

* Package root,
* Workspace root,
* VCS repository, and
* Package repository/index, like crates.io.

Fixes <https://github.com/rust-lang/cargo/issues/4367>

[docs] Wrap long lines

[docs] Update language regarding repository vs package root

In the current documentation, there are many places with language that
assumes a *repository* (as in VCS) is the same as *package root* (the
directory where `Cargo.toml` sits).

With the new workspace features, this is far from true now, and the
inaccurate language makes it difficault for newbies or developers
without much familiarity with the cargo internals with authoring their
manifest files.

This diff tries to use the right terms for places any of these concepts
is referred to:

* Package root,
* Workspace root,
* VCS repository, and
* Package repository/index, like crates.io.

[cargo_new] Hint to use `cargo init` on existing dir

The `new` command always expects a non-existing path. The `init` command
always expects an existing path. Therefore, it makes sense to hint to
use `init` if the pre-condition to `new` is not satisfied.

Fixes <https://github.com/rust-lang/cargo/issues/4366>

[bin/init] Update USAGE

Auto merge of #4372 - frankmcsherry:patch-1, r=alexcrichton

Modernize explanation of `build.rs`

After some recent head-banging, it was explained to me that `"build.rs"` is a special file name, and that Cargo will build such a file even if you do not expect it to, for example if `Cargo.toml` contains the commented out line

```
#build = "build.rs"
```

Some experimentation lead me to the described behavior, but if it misses any subtle details please feel free to correct. The old text is incorrect, though.

Auto merge of #4369 - Wallacoloo:specifying-dependencies-typo, r=alexcrichton

Fix repeated "to to" in Specifying Dependencies

Fixes the typo:

> [...] boil down to the ability to **to** work with a crate [...]

Modernize explanation of `build.rs`

After some recent head-banging, it was explained to me that `"build.rs"` is a special file name, and that Cargo will build such a file even if you do not expect it to, for example if `Cargo.toml` contains the commented out line

```
#build = "build.rs"
```

Some experimentation lead me to the described behavior, but if it misses any subtle details please feel free to correct. The old text is incorrect, though.

Fix repeated "to to" in Specifying Dependencies

Auto merge of #4355 - pornel:docs, r=alexcrichton

Expand profile settings descriptions

Flags were described in the format `foobar = true # controls foobar, passes '--foobar' flag`, but this doesn't actually say what `foobar` does. I've expanded the descriptions a little to provide more context.

Auto merge of #4360 - sid0:members, r=matklad

workspace: export Members as part of the public interface

This struct is returned by `Workspace::members`, but there's no way to
refer to it since it's not exported.

workspace: export Members as part of the public interface

This struct is returned by `Workspace::members`, but there's no way to
refer to it since it's not exported.

Auto merge of #4359 - sid0:debug, r=alexcrichton

derive Debug for more data structures

This makes it easier for wrappers around Cargo to inspect the contents
of these data structures.

derive Debug for more data structures

This makes it easier for wrappers around Cargo to inspect the contents
of these data structures.

Auto merge of #4237 - vignesh-sankaran:blacklist-url-doc, r=carols10cents

Added documentation blacklisting notes

Added notes regarding [documentation domain blacklisting](https://github.com/rust-lang/crates.io/issues/626) on crates.io

Expand profile settings descriptions

Auto merge of #4345 - froydnj:use-noop-more, r=alexcrichton

use Work::noop() more

Using Work::noop() in these places is clearer.

Auto merge of #4344 - froydnj:cargo-rustc-spelling-fixes, r=alexcrichton

fix some spelling errors in cargo_rustc

use Work::noop() more

Using Work::noop() in these places is clearer.

Auto merge of #4342 - sid0:hgignore, r=alexcrichton

new: fix hgignore for real

There was an attempt to fix hgignore in #4158, but unfortunately the fix
was incorrect -- hgignore glob syntax is not in fact identical to
gitignore syntax.

To get rooted ignores in Mercurial we must use the regex-based syntax,
so we have no choice but to define a separate set of ignores.

fix some spelling errors in cargo_rustc

Auto merge of #4343 - sid0:debug, r=alexcrichton

derive/impl Debug for a few data structures

I found this to be extremely useful while trying to write a wrapper on
top of cargo.

derive/impl Debug for a few data structures

I found this to be extremely useful while trying to write a wrapper on
top of cargo.

new: fix hgignore for real

There was an attempt to fix hgignore in #4158, but unfortunately the fix
was incorrect -- hgignore glob syntax is not in fact identical to
gitignore syntax.

To get rooted ignores in Mercurial we must use the regex-based syntax,
so we have no choice but to define a separate set of ignores.

Auto merge of #4340 - debris:fixed_4310, r=alexcrichton

Fixed #4310, print useful message when dependency contains a malformed Cargo.toml

fixed #4310

Removed redundant unwrap

fixed #4310, print useful information when git Cargo.toml is malformed

Auto merge of #4338 - GuillaumeGomez:lint, r=alexcrichton

Update to rc error chain

In order to make https://github.com/rust-lang/rust/pull/43009 works, I needed some small updates that are now in error-chain.

Update error-chain dep for crates-io as well

Update to rc error chain

Auto merge of #4216 - durka:install-multi, r=alexcrichton

cargo install multiple crates

rust-lang-nursery/rustup.rs#986 for `cargo install`

Revives #2601 @pwoolcoc, replaces #3075 @esclear, closes #2585 @kindlychung @cyplo

Avoids the sticking point of the previous two PRs (multiple registry updates) by threading through a first-run boolean flag to decide whether `select_pkg` needs to call `source.update()`.

There is still the issue that flags such as `--git` and `--vers` are "global" to the multiple packages you may be installing. The workaround is just to run `cargo install` separately. In the future we could add syntax like `cargo install foo=1.0 bar=2.5 quux=git://github.com/durka/quux#dev-branch` or something.

Auto merge of #4335 - debris:rebased_4021, r=matklad

Rebased and fixed 4025: Apply --all if workspace is virtual

- fixes #4021
- rebased #4025
- fixed issue issue described by @matklad in https://github.com/rust-lang/cargo/pull/4025#pullrequestreview-40660570
- added test `build_virtual_manifest_one_project` which covers the fix

Updated build_virtual_manifest_one_project test

Fixed test_virtual_manifest_all_implied test

Fixed bench_virtual_manifest_all_implied test

Proper building virtual manifest with -p flag

Adds tests

Apply --all if workspace is virtual

Should help close #4021

Auto merge of #4331 - joshtriplett:fix-notices, r=matklad

Remove inaccurate (misattributed) copyright notices

As discussed with @aturon, who took it to @rust-lang/core.

LICENSE-MIT and one source file contain inaccurate copyright notices of
the form "Copyright (c) (years) The Rust Project Developers", which
implies that an entity called "The Rust Project Developers" holds
copyrights in Rust. Rust contributors retain their copyrights, and do
not assign them to anyone by contributing.  Remove the inaccurate
notices.

Cleaned up notes about documentation blacklisting

Added documentation blacklisting notes

Remove inaccurate (misattributed) copyright notices

LICENSE-MIT and one source file contain inaccurate copyright notices of
the form "Copyright (c) (years) The Rust Project Developers", which
implies that an entity called "The Rust Project Developers" holds
copyrights in Rust. Rust contributors retain their copyrights, and do
not assign them to anyone by contributing.  Remove the inaccurate
notices.

Auto merge of #4329 - behnam:dev, r=matklad

[doc/manifest] Add build-dependencies to Dependency sections list

The `[build-dependencies]` section was missing from the Dependency
sections list, making it hard to find its existence and link to
`specifying-dependencies` page. This puts the section title in the list,
so a normal find works.

Background: https://github.com/rust-lang/cargo/issues/4309#issuecomment-317965620

[doc/manifest] Add build-dependencies to Dependency sections list

The `[build-dependencies]` section was missing from the Dependency
sections list, making it hard to find its existence and link to
`specifying-dependencies` page. This puts the section title in the list,
so a normal find works.

Background: https://github.com/rust-lang/cargo/issues/4309#issuecomment-317965620

nonzero exit code if some crates didn't install

use top level err handling

don't continue if single install errors

Auto merge of #4317 - iKevinY:options-unit-struct, r=matklad

Convert Options struct into unit struct

Unit struct support was added in Docopt 0.8.1 via docopt/docopt.rs#217. Fixes #4174.

Convert Options struct into unit struct

Unit struct support was added in Docopt 0.8.1 via
docopt/docopt.rs#217. Fixes #4174.

Auto merge of #4305 - nodakai:fix-4278-tests-on-empty-ld_library_path, r=matklad

tests/build.rs: remove empty components from LD_LIBRARY_PATH to fix #4278

The tests intended to ensure Cargo wouldn't mistakenly add empty components to `LD_LIBRARY_PATH` (or its equivalents) but they could fail when the test runner already had empty components in `LD_LIBRARY_PATH` from their environment.

tests/build.rs: remove empty components from LD_LIBRARY_PATH

This is to amend PR #4278

The tests intended to ensure Cargo wouldn't mistakenly add empty
components to LD_LIBRARY_PATH (or its equivalents) but they could fail
when the test runner already had empty components in LD_LIBRARY_PATH
from their environment.

Signed-off-by: NODA, Kai <nodakai@gmail.com>

Auto merge of #4303 - bluetech:doc-env-cfg, r=alexcrichton

Document the CARGO_CFG_* environment variables - fixes #4302

In order to keep the paragraph brief, I have omitted:

- examples of actual variable names - though adding those might help google.
- mention of `rustc --print cfg` - I think the link to the reference is enough.
- reference to the [RFC](https://github.com/rust-lang/rfcs/blob/master/text/1721-crt-static.md#lowering-cfg-values-to-cargo-build-script-environment-variables) - I don't think it's needed.

Auto merge of #4308 - alexcrichton:bump, r=alexcrichton

Bump to 0.22.0 and update deps

Work around AppVeyor for now

Bump to 0.22.0 and update deps

Document the CARGO_CFG_* environment variables - fixes #4302

Auto merge of #4299 - brson:home, r=alexcrichton

Replace homedir definition with home crate

This pulls out the definitions of these functions into a single crate, [home](https://github.com/brson/home), to act as their canonical definition.

This has a bigger impact on rustup, since it is using  definitions of all these things, where cargo is just defining cargo_home in terms of std's wrong home_dir.

The [definition of cargo_home](https://github.com/brson/home/blob/master/src/lib.rs#L179) used here is slightly different than what cargo has today in that it contains a bit of defensive code to avoid allowing `CARGO_HOME` to be set to `.multirust/cargo` - this is backwards compatibility code from rustup.

Auto merge of #4298 - behnam:ignore, r=alexcrichton

[src/doc/manifest] Add section on Migrating to `gitignore`-like pattern matching

Tracking issue: https://github.com/rust-lang/cargo/issues/4268

[src/doc/manifest] Add section on Migrating to `gitignore`-like pattern matching

Tracking issue: https://github.com/rust-lang/cargo/issues/4268

Replace homedir definition with home crate

Auto merge of #4123 - alexcrichton:augment, r=matklad

Implement the [patch] section of the manifest

This is an implementation of [RFC 1969] which adds a new section to top-level
manifests: `[patch]`. This section allows you to augment existing sources with
new versions of crates, possibly replacing the versions that already exist in
the source. More details about this feature can be found in the RFC itself.

[RFC 1969]: https://github.com/rust-lang/rfcs/pull/1969

Implement the [patch] section of the manifest

This is an implementation of [RFC 1969] which adds a new section to top-level
manifests: `[patch]`. This section allows you to patch existing sources with
new versions of crates, possibly replacing the versions that already exist in
the source. More details about this feature can be found in the RFC itself.

[RFC 1969]: https://github.com/rust-lang/rfcs/pull/1969

Auto merge of #4270 - behnam:ignore, r=alexcrichton

[sources/path] Add gitignore-like pattern matching and warn on mismatches

Add gitignore-like pattern matching logic to `list_files()` and throw
warnings for paths getting different inclusion/exclusion results from
the old and the new methods.

Migration Tracking: <#4268>

Auto merge of #4295 - xftroxgpx:use_full_path_in_warning_for_issue4293, r=matklad

Fixes #4293

locally tested with:
`cargo test legacy_ many_crate_types_old_style_lib_location`

remove leftover { }

Fixes #4293

locally tested with:
cargo test legacy_ many_crate_types_old_style_lib_location

Auto merge of #4292 - xftroxgpx:unconfuse_bin_name, r=matklad

quote the binary name in the warning

to avoid confusion when the warning happens inside a workspace, example:
pwd is rustlearnage/rust_books/1_first_edition/closures/
and `cargo run` shows:
```
warning: path `src/main.rs` was erroneously implicitly accepted for binary match,
please set bin.path in Cargo.toml
```
(note that 'match' is not quoted, so it's confusing)

the 'match' project that it's referring to is in rustlearnage/rust_books/1_first_edition/match/
(note: Cargo.toml [workspace] is in rustlearnage/ )

quoting library and benchmark names in warning

quote the binary name in the warning

to avoid confusion when the warning happens inside a workspace, example:
pwd is rustlearnage/rust_books/1_first_edition/closures/
and `cargo run` shows:

warning: path `src/main.rs` was erroneously implicitly accepted for binary match,
please set bin.path in Cargo.toml

(note that 'match' is not quoted, so it's confusing)

the 'match' project that it's referring to is in rustlearnage/rust_books/1_first_edition/match/
(note: Cargo.toml [workspace] is in rustlearnage/ )

[sources/path] Add gitignore-like pattern matching and warn on mismatches

Add gitignore-like pattern matching logic to `list_files()` and throw
warnings for paths getting different inclusion/exclusion results from
the old and the new methods.

Migration Tracking: <https://github.com/rust-lang/cargo/issues/4268>

Auto merge of #4289 - johnthagen:usage-fix, r=alexcrichton

Add "uninstall" command to cargo --help

Closes #4285

Auto merge of #4288 - matklad:bench-bench, r=alexcrichton

Allow `src/bench.rs` as a benchmark for legacy reasons

closes #4287
cc #3947

Add "uninstall" command to cargo --help

Auto merge of #4286 - cbcoutinho:master, r=matklad

Fix typo in zsh _cargo completions

None

Allow `src/bench.rs` as a benchmark for legacy reasons

closes #4287
cc #3947

Fix typo in zsh _cargo completions

Auto merge of #4279 - tee-too:fix-3499, r=alexcrichton

Fix support of `[target.'cfg(...)']` syntax for rustc and rustdoc flags

Support of `[target.'cfg(...)']` for rustc and rustdoc flags is buggy.
This adds meaningful tests and fixes the issue.

Fixes #3499

Fix support of `[target.'cfg(...)']` syntax for rustc and rustdoc flags

Support of `[target.'cfg(...)']` for rustc and rustdoc flags is buggy.
This adds meaningful tests and fixes the issue.

Fixes #3499

Auto merge of #4278 - mbrubeck:path, r=alexcrichton

Don't push empty paths in LD_LIBRARY_PATH

Fixes #4277.  Note: I haven't written a test for this fix yet.

Don't push empty paths in LD_LIBRARY_PATH

Fixes #4277.

Auto merge of #4275 - alexcrichton:revert-makeflags, r=alexcrichton

Don't set MAKEFLAGS for build scripts

Closes #4156
Closes rust-lang/rust#42635

Don't set MAKEFLAGS for build scripts

Closes #4156
Closes rust-lang/rust#42635

add test for failing package

use shell helpers to condense output

only print PATH warning once

Auto merge of #4274 - sfackler:patch-1, r=alexcrichton

Add CircleCI badge to manifest reference

rust-lang/crates.io#807

Add CircleCI badge to manifest reference

rust-lang/crates.io#807

Auto merge of #4272 - behnam:doc, r=alexcrichton

[doc/crates-io] Add text about packaging tests/benches

Fixes <https://github.com/rust-lang/cargo/issues/4262>

[doc/crates-io] Add text about packaging tests/benches

Fixes <https://github.com/rust-lang/cargo/issues/4262>

Auto merge of #4259 - matklad:conventions, r=alexcrichton

Conventions

r? @alexcrichton

I'd love to refactor our handing of inferring targets by convention, because it is super difficult to understand, and quite probably contains a couple of unintended/undocumented conventions (like `src/foo.rs` being a binary `foo` if there's no library in the package).

As a first step, I've just moved all the logic (600 loc) to a separate file from the `toml.rs` file, which used to be just **huge** (we even use in in IntelliJ Rust for performance testing :) ), and now it is "only" just above 1kloc :)

Auto merge of #4267 - natboehm:add-index-flag-search, r=matklad

Replace `cargo search --host` with `cargo search --index`

This change makes the command consistent with other versions of the same flag. `cargo search --host` is still supported, as well as `cargo search --index <index> --host <host>` in case of confusion. The three commands currently supported should produce the same behavior. All instances of `--host` are currently marked as deprecated, or for reuse as mentioned in issue #4208.

Fixes #4219

Modified tests to use '--index' instead of '--host', added test to support using '--host' with warning message, added test to support using both '--host' and '--index' with warning message for host

rename '--host' to '--index', add comments marked as TODO where host should be removed/repurposed

Auto merge of #4266 - nrc:force, r=alexcrichton

Add a way to force a unit of work to always be rebuilt

Add a way to force a unit of work to always be rebuilt

Rearrange code

Simplify

Use Path::join instead of `/`

While `/` do work on windows, the resulting path will have literal `/`
in its display.

Fix more tests